Skip to content

Conversation

compnerd
Copy link
Member

This series cleans up the vast majority of the @testable importing in the tests. The one that remains is for X509 which is external to this package. I intend to introduce a SPI for that API surface to help remove the last of the @testable importing.

@compnerd compnerd force-pushed the testable branch 2 times, most recently from d27a86c to 4ea6c2c Compare June 18, 2025 00:39
@compnerd
Copy link
Member Author

@swift-ci please test

@bkhouri
Copy link
Contributor

bkhouri commented Jun 18, 2025

@swift-ci test

@bkhouri
Copy link
Contributor

bkhouri commented Jun 18, 2025

This change may conflict with #8821

@compnerd
Copy link
Member Author

Yeah, I think that we should just go for this single shot - the changes are all mechanical, but the CI times are quite long. I'm happy to hand off the set if someone wants to help drive merging the full set into the tree split up.

@compnerd
Copy link
Member Author

There seem to be a number of failures on Linux:

[2025-06-18T17:09:34.681Z]     error: posix_spawn error: No such file or directory (2), `["/tmp/Miscellaneous_TestDiscovery_Simple.GsF1UV/Miscellaneous_TestDiscovery_Simple/.build/x86_64-unknown-linux-gnu/Products/Debug-linux/SimpleTests.xctest"]`
[2025-06-18T17:09:34.681Z]     error: posix_spawn error: No such file or directory (2), `["/tmp/Miscellaneous_TestDiscovery_Simple.GsF1UV/Miscellaneous_TestDiscovery_Simple/.build/x86_64-unknown-linux-gnu/Products/Debug-linux/SimpleTests.xctest", "--package-path", "/tmp/Miscellaneous_TestDiscovery_Simple.GsF1UV/Miscellaneous_TestDiscovery_Simple", "--configuration", "debug", "--build-system", "swiftbuild", "--skip-build", "--enable-code-coverage", "--testing-library", "swift-testing"]`

@compnerd
Copy link
Member Author

@swift-ci please test Windows platform

@compnerd
Copy link
Member Author

@swift-ci please test macOS platform

@compnerd
Copy link
Member Author

@swift-ci please test Linux platform

@compnerd
Copy link
Member Author

@swift-ci please test

@compnerd
Copy link
Member Author

@swift-ci please test Windows platform

This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
This extends the visibility of some of the private implementation to the
package level to allow use for testing without `@testable` imports.
Use the newly extended APIs for testing and use a non-testable import of
the package, avoiding a recompilation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants